-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implementation of a new EBL model from Finke et al. 2022. #465
Conversation
A new class -- inhereting from the TabularPhotonField class -- was created in the PhotonBackground.h file to fully implement the new EBL model. More over the model was added to various test calculations for every interaction in the testInteraction.cpp script.
For the record, I've been waiting for the changes in the CRPropa3-data repo to check and merge both together. |
Hi @rafaelab, @Yannick-Schmidt, @JulienDoerner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not see any serious issue here, only some small style comments.
I will aprove it anyway so that it can be merged afterwards.
@@ -40,6 +40,8 @@ TEST(ElectronPairProduction, allBackgrounds) { | |||
epp.setPhotonField(irb); | |||
irb = new IRB_Stecker16_lower(); | |||
epp.setPhotonField(irb); | |||
irb = new IRB_Finke22(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whitespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not creating a new commit just to fix the whitepace, but we should probably do this at some point in a consistent way for all files.
@@ -718,6 +724,8 @@ TEST(EMPairProduction, allBackgrounds) { | |||
em.setPhotonField(ebl); | |||
ebl = new IRB_Stecker16_lower(); | |||
em.setPhotonField(ebl); | |||
ebl = new IRB_Finke22(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it called EBL here and IRB else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that depends on who implemented the test in the first place. Could be cleaned up in the future.
This PR adds the recent model from Finke et al. 2022 to the existing selection of custom photon fields. It is related to the corresponding pull request in the CRPropa-data repository.
The implementation includes the ...
TabularPhotonField
base class (in thePhotonBackground.h
file).testInteraction.cpp
).